Skip to content

fix(deps): align NumPy requirement with package metadata - #21

Merged
yh-vino merged 1 commit into
HYGON-AI:mainfrom
yh-vino:fix/align-numpy-requirement
Oct 9, 2026
Merged

yh-vino merged 1 commit into
HYGON-AI:mainfrom
yh-vino:fix/align-numpy-requirement

Conversation

@yh-vino

@yh-vino yh-vino commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

问题与修改

requirements.txt 将 NumPy 固定为 1.23.5,而 setup.py 声明为 numpy>=1.23.0,导致两种安装方式的依赖约束不一致。按 requirements 安装时可能将现有、满足组件声明的 NumPy 版本替换为 1.23.5。

将 requirements.txt 中该项改为 numpy>=1.23.0,与组件安装元数据一致。本次仅修改这一行,其他依赖及 CI 镜像的固定版本保持不变。

验证

  • 已核对 NumPy 约束与 setup.py 一致。
  • git diff --check 通过。
  • 未运行 HCU 测试或 NumPy 多版本兼容测试;本次统一已有依赖声明,不代表新增版本兼容性已经验证。

@hygon-ai-reviewer hygon-ai-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI Review

🟢 未发现有明确证据的问题

在已提供并完成审查的 diff 范围内未形成可确认问题。

变更概览

本次变更主要包含:

  • 将 requirements.txt 中 NumPy 约束由固定 numpy==1.23.5 放宽为 numpy>=1.23.0,使其与包元数据中对 NumPy 的声明保持一致
文件审查摘要
文件 变更 审查结果
requirements.txt 修改 · +1/-1 —
审查信息
  • 变更统计:1 个文件,+1/-1。
  • 覆盖情况:共 1 个文件,已完整审查 1 个。
  • 候选问题:0 项;证据复核过滤:0 项;发布前敏感信息保护:0 项。
  • 本服务以 GitHub 提供的 PR diff 为审查主体,PR 描述与按相关性选取的仓库片段仅用于核验;未执行代码或重跑测试,结论仍需维护者核验。

@yh-vino
yh-vino requested a review from WXuan-X October 9, 2026 07:09
@yh-vino
yh-vino merged commit e43dc7c into HYGON-AI:main Oct 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants